Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add optional logout menu entry. #217

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gjherbiet
Copy link

This pull request add and optional logout menu entry, configured by two configuration variables in the [web] section:

  • logout_url: the target URL of the logout menu entry
  • logout_text: the displayed text in the logout menu entry

Please note that the link will not perform any logout by itself. I is the responsibility of the system administrator to make something useful out of the link.

For instance, when running dns-ui behind Nginx as a reverse proxy (using the auth_request module) , with oauth2-proxy and Keycloak OIDC as authentication client, the following snippets can be useful:

In Nginx config:

location /logout {
	return 302 http://<url_of_your_openauth2_proxy_instance>/sign_out?rd=<url_of_page_to_display_after_logout>;
}

In oauth2-proxy config:

backend_logout_url="http://<url_of_your_keycloak_instance>/realms/<realm_name>/protocol/openid-connect/logout?id_token_hint={id_token}"

@gjherbiet gjherbiet force-pushed the feature/logout-link branch 2 times, most recently from 53b50fb to 19a1d84 Compare October 12, 2024 16:06
@gjherbiet
Copy link
Author

This potentially closes #20.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant